@bhamail Hi Dan. Don't suppose we could merge this?

It fails the very stale CI, but that will take a while to fix. It fixes the R CMD checks on CRAN - which is the main one.

The security warnings are around an old requirements.txt that we don't use for installation - just parsing

@bhamail Hi Dan. Don't suppose we could merge this?

It fails the very stale CI, but that will take a while to fix. It fixes the R CMD checks on CRAN - which is the main one.

The security warnings are around an old requirements.txt that we don't use for installation - just parsing

@csgillespie Hi Colin! Yeah, that sound good to me. Happy to merge this puppy.

For kicks, I tried to get CI to run locally and ran into the error below:

====>> Build package
  #!/bin/bash -eo pipefail
R CMD build .
* checking for file ‘./DESCRIPTION’ ... OK
* preparing ‘oysteR’:
* checking DESCRIPTION meta-information ... OK
* installing the package to build vignettes
      -----------------------------------
ERROR: dependency ‘rjson’ is not available for package ‘oysteR’
* removing ‘/tmp/RtmpyP9wMn/Rinst3438e487ba/oysteR’
      -----------------------------------
ERROR: package installation failed
Error: 
Exited with code exit status 1

Any hints on how to resolve this? I'm happy to try and help fixing the CI issue after this PR is merged, but would appreciate your help hammering on it.

@collinpeters Oh, also looks like I will need to update changes from master into this branch. Duck and cover while I do that first.

@collinpeters OK, I made some changes to hopefully get CI further along the path to happy. Will merge this puppy. Please take a look at main afterwards and see if we can get a happy build in circleci and/or githubActions.